Skip to content

[fix] Keep Launch visible and group Reset below settings - #187

Merged
themartiano merged 10 commits into
omacom:mainfrom
Fail-Safe:codex/keep-launch-visible
Sep 21, 2026
Merged

themartiano merged 10 commits into
omacom:mainfrom
Fail-Safe:codex/keep-launch-visible

Conversation

@Fail-Safe

@Fail-Safe Fail-Safe commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

The expanded start menu can push Launch below the visible area. Keep Launch in a fixed footer while permissions and settings scroll above it, and size the window to its laid-out content up to the display’s available height. Preserve the settings’ scroll position when sizing the window.

Reset lives in its own section below Immersive, with an icon, a Factory reset title, and a brief description. Its button uses red text and the standard border at rest, then fills red on hover. The section stays in the scrolling content. Return-key launch, typed reset confirmation, permissions, and VM settings retain their existing behavior.

Validation:

  • Full make test passed after merging current main: 275 native tests, 153 guest Python tests, and the shell/storage checks. The optional virtio-pinch ABI and staged-QEMU lock-inheritance checks were skipped because their binaries were unavailable.
  • The start-menu regression test verifies that Launch stays outside the scroll view and Reset follows the integrations inside it, with Return-key launch preserved.
  • The rebuilt local app was opened and visually checked for the fixed Launch footer, the Reset section below Immersive, and the final button appearance.
  • git diff --check passed.

@themartiano themartiano changed the title [fix] Keep Launch visible with a compact action area [fix] Keep Launch visible and group Reset below settings Sep 21, 2026
@themartiano
themartiano merged commit 9fe766c into omacom:main Sep 21, 2026
1 check passed
@Fail-Safe
Fail-Safe deleted the codex/keep-launch-visible branch September 23, 2026 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants